webrtc wpt: add cleanup to close peerconnections adds a pc.close() to all RTCPeerConnections in promise_test and async_test. This is codemod-powered by this: https://github.com/fippo/webrtc-codemods/blob/e844b2467cbb76a231c113366b2451cf248f53bc/wpt-rtcpeerconnection-no-helpers BUG=836871 Change-Id: Iee84ef938bf477a55612b6012c6de464c6aea55b Reviewed-on: https://chromium-review.googlesource.com/1049983 Commit-Queue: Henrik Boström <hbos@chromium.org> Reviewed-by: Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#565591} diff --git a/webrtc/RTCPeerConnection-peerIdentity.html b/webrtc/RTCPeerConnection-peerIdentity.html index 68d9f20..64ad212 100644 --- a/webrtc/RTCPeerConnection-peerIdentity.html +++ b/webrtc/RTCPeerConnection-peerIdentity.html
@@ -56,8 +56,11 @@ */ promise_test(t => { const pc1 = new RTCPeerConnection(); + t.add_cleanup(() => pc1.close()); const pc2 = new RTCPeerConnection(); + t.add_cleanup(() => pc2.close()); + const port = window.location.port; const [idpDomain] = getIdpDomains(); const idpHost = hostString(idpDomain, port); @@ -92,10 +95,13 @@ const idpHost = hostString(idpDomain, port); const pc1 = new RTCPeerConnection(); + t.add_cleanup(() => pc1.close()); const pc2 = new RTCPeerConnection({ peerIdentity: `bob@${idpDomain}` }); + t.add_cleanup(() => pc2.close()); + pc1.setIdentityProvider(idpHost, { protocol: 'mock-idp.js', usernameHint: `alice@${idpDomain}` @@ -131,10 +137,13 @@ const idpHost = hostString(idpDomain, port); const pc1 = new RTCPeerConnection(); + t.add_cleanup(() => pc1.close()); const pc2 = new RTCPeerConnection({ peerIdentity: `alice@${idpDomain}` }); + t.add_cleanup(() => pc2.close()); + // Ask mockidp.js to return custom contents in validation result pc1.setIdentityProvider(idpHost, { protocol: 'mock-idp.js?validatorAction=return-custom-contents&contents=bogus', @@ -167,10 +176,13 @@ const idpHost1 = hostString(idpDomain1, port); const pc1 = new RTCPeerConnection(); + t.add_cleanup(() => pc1.close()); const pc2 = new RTCPeerConnection({ peerIdentity: `alice@${idpDomain2}` }); + t.add_cleanup(() => pc2.close()); + // mock-idp.js will return assertion of domain2 identity // with domain1 in the idp.domain field pc1.setIdentityProvider(idpHost1, { @@ -220,10 +232,13 @@ const idpHost = hostString(idpDomain, port); const pc1 = new RTCPeerConnection(); + t.add_cleanup(() => pc1.close()); const pc2 = new RTCPeerConnection({ peerIdentity: `alice@${idpDomain}` }); + t.add_cleanup(() => pc2.close()); + // Ask mock-idp.js to throw error during validation, // i.e. during pc2.setRemoteDescription() pc1.setIdentityProvider(idpHost, { @@ -263,8 +278,11 @@ */ promise_test(t => { const pc1 = new RTCPeerConnection(); + t.add_cleanup(() => pc1.close()); const pc2 = new RTCPeerConnection(); + t.add_cleanup(() => pc2.close()); + const port = window.location.port; const [idpDomain] = getIdpDomains(); const idpHost = hostString(idpDomain, port); 